Skip to content

sshutil: suppress warnings when SSH lacks GSSAPI #3637

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

kachick
Copy link
Contributor

@kachick kachick commented Jun 19, 2025

Background

Starting with NixOS 24.11, NixOS and nixpkgs no longer enable GSSAPI support in the openssh package by default.
See:

Fixes #3198

How

The ssh -G nothing approach requires generating a random string and making an additional call to an external ssh command.
Instead, I combined this change into the current ssh -V implementation. Is this approach acceptable?

AkihiroSuda
AkihiroSuda previously approved these changes Jun 19, 2025
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@AkihiroSuda AkihiroSuda added this to the v1.1.2 milestone Jun 19, 2025
@AkihiroSuda AkihiroSuda requested a review from afbjorklund June 19, 2025 08:35
@kachick kachick marked this pull request as draft June 19, 2025 17:00
@kachick kachick force-pushed the consider-ssh-unsupported-gssapiauthentication branch 2 times, most recently from 9e9e67b to 1fbf86d Compare June 19, 2025 17:18
@kachick kachick marked this pull request as ready for review June 19, 2025 17:24
@kachick kachick marked this pull request as draft June 19, 2025 17:52
Co-authored-by: Oleksandr Redko <[email protected]>
Signed-off-by: Kenichi Kamiya <[email protected]>
Signed-off-by: Oleksandr Redko <[email protected]>
@kachick kachick force-pushed the consider-ssh-unsupported-gssapiauthentication branch from ab4f701 to 8b0f86e Compare June 19, 2025 17:56
@kachick kachick marked this pull request as ready for review June 19, 2025 17:59
@AkihiroSuda AkihiroSuda merged commit 29c614c into lima-vm:master Jun 20, 2025
38 checks passed
@kachick
Copy link
Contributor Author

kachick commented Jun 20, 2025

Thank you for reviewing and merging 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatically detect if GSSAPIAuthentication is available or not
3 participants